Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: spin-up ephemeral ESS clusters to run some system tests #2551

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

v1v
Copy link
Member

@v1v v1v commented Feb 14, 2025

What

Using the production environment for this makes sense to ensure we are testing against released APM features.

In the subsequent tests, we'd need to access the APM endpoint and ApiKey to push APM data from the test applications. In the test projects, we'd need access to an API key with permissions to query Elasticsearch observability data streams to validate the received data.

As a stretch feature, although not critical from day one, it would be helpful to have programmatic access to query APM server logs from the project to log any errors that might occur during intake. This is one advantage to the in-memory mocked APM server we have today.

Summary steps:

  1. Create an Observability Serverless project as part of the CI workflow
  2. Provide access to specific credentials for that project in the following workflow steps (test projects)
  3. Destroy the project when the CI has finished

Tasks

@v1v v1v self-assigned this Feb 14, 2025
v1v and others added 4 commits February 17, 2025 17:31
probably we can use the oblt-cli binary in a script instead of using GH actions
chore

run after
@v1v v1v changed the title ci: spinup ESS cluster for the latest changes in main ci: spin-up ephemeral ESS clusters to run some system tests Feb 17, 2025
# NOTE: you can now use the cluster credentials to run your tests
# please see https://github.com/elastic/oblt-actions/tree/v1/oblt-cli/cluster-credentials#exported-environment-variables
# for the environment variables that are set by this action
- run: echo "TBC"
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the step to honour the created ephemeral ESS projects

@stevejgordon, what are your thoughts?

@v1v v1v marked this pull request as ready for review February 18, 2025 13:39
@v1v v1v requested a review from a team February 18, 2025 13:40
permissions: >-
{
"contents": "write",
"pull_requests": "read"
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"pull_requests": "read"
"pull_requests": "read",
"checks": "read"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant